-
Notifications
You must be signed in to change notification settings - Fork 219
Onboard PostgreSQL 18 #641
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
|
[test-all] |
Pull Request validationFailed🔴 Approval - missing or changes were requested Success🟢 CI - All checks have passed |
Testing Farm results
|
|
[test-all] |
|
See take an inspiration from #548. For me it is better to review this huge change. |
90c5f6f to
31c20e3
Compare
cf5c2c3 to
bbbdf41
Compare
|
[test-all] |
|
Depends on #643 |
|
[test-all] |
bbbdf41 to
eb0e20b
Compare
|
[test-all] |
|
Fedora postgresql18 test fails because f43 has the wrong pgaudit. Fix here: https://src.fedoraproject.org/rpms/postgresql18-pgaudit/pull-request/3 CentOS Stream 9 tests fail because the postgresql:18 module isn't there yet. |
eb0e20b to
bbdfc4c
Compare
|
[test-all] |
a35ca46 to
1ee57a6
Compare
|
[test-all] |
phracek
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please address my comments. I will look at the tests in the meanwhile.
phracek
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update also build-and-push file in .github/workflows directory.
Add URI encoding of user and db name in tests that use the connection URI to access a container, test that it works with both spaces and special characters. This is because postgres 18 does not accept spaces in the connection URI but earlier versions do.
Revert this when imagestreams are available
c9s should be added when rpms become available
1ee57a6 to
97ed698
Compare
|
[test-all] |
PTAL at how I changed the dist-gen source files. I tried to be as non-intrusive as possible.